Skip to content

Conversation

@minminlittleshrimp
Copy link
Collaborator

Fix debian build local

@minminlittleshrimp minminlittleshrimp force-pushed the dlt_dev_debian_update branch 5 times, most recently from c843fa1 to 8009bf5 Compare August 3, 2025 07:17
@minminlittleshrimp
Copy link
Collaborator Author

minminlittleshrimp commented Aug 3, 2025

This needs merging first: #761
Or simply apply this:

index e705c3e..d226daf 100644
--- a/src/offlinelogstorage/dlt_offline_logstorage_behavior.c
+++ b/src/offlinelogstorage/dlt_offline_logstorage_behavior.c
@@ -1108,7 +1108,7 @@ int dlt_logstorage_prepare_on_msg(DltLogStorageFilterConfig *config,
                 if ((config->sync == DLT_LOGSTORAGE_SYNC_ON_MSG) ||
                     (config->sync == DLT_LOGSTORAGE_SYNC_UNSET)) {
                     if (config->gzip_compression == DLT_LOGSTORAGE_GZIP_ON) {
-                        if (fsync(fileno(config->gzlog)) != 0) {
+                        if (fsync(config->fd) != 0) {
                             if (errno != ENOSYS) {
                                 dlt_vlog(LOG_ERR, "%s: failed to sync gzip log
 file\n", __func__);
                             }

Build result:
0001_dlt_v2.18.10_debian_local_build.txt

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

@minminlittleshrimp minminlittleshrimp force-pushed the dlt_dev_debian_update branch 2 times, most recently from e3e9edf to 791a33e Compare August 3, 2025 07:35

This comment was marked as outdated.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the Debian packaging configuration to fix local build issues for the DLT daemon project. The changes modernize the Debian build system and address installation path inconsistencies.

Key changes include:

  • Modernization of Debian packaging from debhelper 9 to debhelper-compat 13
  • Fixes to systemd service file installation paths to use the correct prefix
  • Addition of build configuration files and better handling of systemd version detection

Reviewed Changes

Copilot reviewed 10 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
debian/source/option Adds diff ignore configuration for Debian source package
debian/source/format Sets Debian source package format to native
debian/rules Improves formatting of CMake configuration options
debian/not-installed Lists files intentionally not included in packages
debian/dlt-tools.install Fixes systemd service file paths to use proper prefix
debian/dlt-daemon.install Fixes systemd service file path to use proper prefix
debian/control Updates build dependency to modern debhelper-compat
debian/compat Removes deprecated compat file
debian/changelog Adds changelog entry for version 2.18.10
CMakeLists.txt Adds null check for systemd version detection

Signed-off-by: lum3hc <Minh.LuuQuang@vn.bosch.com>
@lti9hc lti9hc merged commit f1d4f51 into master Aug 26, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Not able to Build DLT Debian package using the dpkg-buildpackage -us -uc

3 participants